Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert amplicon16S_analysis workflow (QIIME) from single-sample to multi-sample #452

Merged
merged 31 commits into from
Feb 16, 2023

Conversation

golu099
Copy link
Contributor

@golu099 golu099 commented Feb 3, 2023

Updating tasks_16S_amplicon to consider multiple files for analysis. Curious to know if theres any suggestion to optimization for this run?

@golu099 golu099 changed the title Fnegrete test QIIME WDL integraiton into Master branch Feb 9, 2023
@golu099
Copy link
Contributor Author

golu099 commented Feb 10, 2023

@dpark01 @tomkinsc do you mind reviewing this to be merged onto main branch?

@dpark01 dpark01 changed the title QIIME WDL integraiton into Master branch Convert amplicon16S_analysis workflow (QIIME) from single-sample to multi-sample Feb 14, 2023
pipes/WDL/tasks/tasks_16S_amplicon.wdl Outdated Show resolved Hide resolved
Copy link
Member

@dpark01 dpark01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small comments and suggestions

pipes/WDL/tasks/tasks_16S_amplicon.wdl Outdated Show resolved Hide resolved
pipes/WDL/tasks/tasks_16S_amplicon.wdl Outdated Show resolved Hide resolved
pipes/WDL/tasks/tasks_16S_amplicon.wdl Outdated Show resolved Hide resolved
pipes/WDL/tasks/tasks_16S_amplicon.wdl Show resolved Hide resolved
pipes/WDL/tasks/tasks_16S_amplicon.wdl Outdated Show resolved Hide resolved
pipes/WDL/tasks/tasks_16S_amplicon.wdl Outdated Show resolved Hide resolved
@golu099 golu099 merged commit aad7bf2 into master Feb 16, 2023
@golu099 golu099 deleted the fnegrete_test branch February 16, 2023 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants